feat: add webapi response changes as of 2026-08-07 by slackapi[bot] · Pull Request #1629 · slackapi/java-slack-sdk

slackapi · 2026-08-07T01:43:10Z

This PR updates the web API client and model based on the production E2E test results.

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

codecov · 2026-08-07T01:50:28Z

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.24%. Comparing base (86ff7ca) to head (854c57e).
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
.../java/com/slack/api/util/json/GsonFileFactory.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1629      +/-   ##
============================================
- Coverage     73.34%   73.24%   -0.10%     
+ Complexity     4528     4522       -6     
============================================
  Files           478      478              
  Lines         14314    14314              
  Branches       1491     1491              
============================================
- Hits          10498    10484      -14     
- Misses         2924     2940      +16     
+ Partials        892      890       -2

Flag	Coverage Δ
jdk-14	`73.24% <0.00%> (-0.10%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

ActionsResponse.Actions: List<String> workObjectRecord SchemasResponse: WorkObjectRecord class (id, appId, objectId, objectType) Actions: WorkObjectRecord inner class with enterprise_search_email_attachment_downloaded Additive, no breaking changes; verified against the live audit API response. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

rtmConnect(token) requests full user info, adding a users.info call between rtm.connect and the websocket. That second HTTPS request may or may not reuse the pooled proxy tunnel, so the proxy occasionally sees a third CONNECT and the callCount assertion flakes between 2 and 3. The test does not use the connected user info, so request it without the extra call to keep the count deterministic at 2. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

zimeg

🌻 A comment as changes happen.

zimeg · 2026-08-08T19:40:58Z

+        // fullUserInfoRequired=false: skip the extra users.info call so the proxy sees exactly
+        // two connections (rtm.connect + websocket), otherwise a third tunnel can be opened
+        try (RTMClient rtm = slack.rtmConnect(rtmBotToken, false)) { // slack-msgs.com


💡 note: Added to reduce flakes of scheduled tests! We now hope to focus on new properties with the test runner.

Detected as a new audit action by the E2E runner (Unknown action detected). Adds the constant alongside the other mcp_slack_*_tool_called entries. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

…tured.list Assert the success-only `message` field on chat.stopStream, and rewrite the workflows.featured.list E2E test to a real add -> list -> assert -> remove flow (resolving a real channel and reading a trigger id from SLACK_SDK_TEST_WORKFLOW_TRIGGER_ID, no-op when unset) so the sample recorder captures featured_workflows. Error-only tests only ever recorded the base error shape, leaving the generated response types thin. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

Use JUnit Assume instead of an early return so the test reports as skipped (not a silent pass) when SLACK_SDK_TEST_WORKFLOW_TRIGGER_ID or a resolvable channel is absent. A bare return counted the test as passed while recording nothing, hiding that the featured_workflows sample never got captured. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

workflows.featured.list resolved an arbitrary channel (first in the list), which returned restricted_action from featuredAdd. Target the workspace #general channel instead, matching how files_Test resolves it, and skip when it isn't found. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

workflows.featured.add/set/remove require a caller with permission to manage workflows in the target channel. A bot token is denied with "restricted_action" even when it holds workflows.templates:write and is a channel member, so the add->list->assert->remove flow must run on the user token. Reproduced across four channels; user token succeeds and list returns the full featured_workflows[].triggers[].{id,title} shape. Skips honestly when SLACK_SDK_TEST_USER_TOKEN is unset. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

pins.list nests the pinned message as a top-level com.slack.api.model.Message, and the response now carries a permalink URL string that the top-level class did not model (permalink existed only on the MessageItem inner class). Under strict unknown-property detection this surfaced as "Unknown property detected: permalink (com.slack.api.model.Message)". ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

…low test findChannelId listed channels with the bot token and a single limit(100) request, so it could miss #general (page truncation) or return one the user token cannot feature in — tripping the Assume gate and silently skipping the featured-workflow flow. List from the user token (the acting identity) and page through next_cursor so the test actually runs and records the featured_workflows shape. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

serialize(File) called context.serialize(src), which re-dispatched to this same registered adapter and recursed until StackOverflowError. That crashed SampleObjects.Json static init, which in turn aborted json-log sample recording for any response containing a nested Message (e.g. chat.stopStream), silently dropping fields like `message` from the generated samples. Serialize as the NormalizedFile subclass (no registered adapter, default reflective serializer), mirroring the existing deserialize() workaround. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

zimeg

📣 Leaving notes on changes to the test runner itself that might be worth more consideration!

I'm eager to land the additions to response for handfuls of method for improved typescript usage downstream!

zimeg · 2026-08-14T21:05:06Z

    @Override
    public JsonElement serialize(File src, Type typeOfSrc, JsonSerializationContext context) {
-        return context.serialize(src);
+        // To prevent StackOverflowError here, serialize as File's subclass (see deserialize above).
+        return context.serialize(src, NormalizedFile.class);
    }
 }


🗣️ note: We workaround the errors of serialization that caused various message properties to be missing due to infinite recursion using the same "NormalizedFile" class technique above.

zimeg · 2026-08-14T21:06:35Z

-                ).get();
-        assertThat(workflowsFeaturedListResponse.getError(), is(notNullValue()));
+    public void workflowsFeaturedList() throws Exception {
+        String triggerId = System.getenv(Constants.SLACK_SDK_TEST_WORKFLOW_TRIGGER_ID);


👁️‍🗨️ note: We build confidence in responses of workflows.featured.* using a user token and "general" channel ID with existing workflow values.

zimeg · 2026-08-14T21:07:01Z

                ));
        assertThat(stops.isOk(), is(true));
        assertThat(stops.getError(), is(nullValue()));
+        assertThat(stops.getMessage(), is(notNullValue()));


👾 note: This adds confidence that the streamed message values are returned and detected as expected here!

zimeg · 2026-08-14T21:08:49Z

    public static final String SLACK_SDK_TEST_CLASSIC_APP_BOT_TOKEN = "SLACK_SDK_TEST_CLASSIC_APP_BOT_TOKEN";

+    // featured workflows tests
+    public static final String SLACK_SDK_TEST_WORKFLOW_TRIGGER_ID = "SLACK_SDK_TEST_WORKFLOW_TRIGGER_ID";


🏁 note: An adjacent variable is used with the entire trigger URL but isn't defined in these constants at this time. A follow up PR plans to document this to latest standards our runner uses!

admin.apps.activities.list responses now include provider_key on the activity payload (observed value "mcp_dcr_notion"), which strict unknown-property detection flagged as "Unknown property detected: provider_key (com.slack.api.model.admin.AppActivity$Payload)". ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

feat: add webapi response changes as of 2026-08-07

42a1976

slackapi Bot added the project:slack-api-model project:slack-api-model label Aug 7, 2026

slackapi Bot requested a review from a team as a code owner August 7, 2026 01:43

slackapi Bot added the semver:patch label Aug 7, 2026

zimeg and others added 6 commits August 7, 2026 14:38

Merge remote-tracking branch 'origin/main' into api

acdeda1

feat: add webapi response changes as of 2026-08-07

6057fa9

feat: add webapi response changes as of 2026-08-08

d71439b

feat: add webapi response changes as of 2026-08-08

a37fb04

zimeg reviewed Aug 8, 2026

View reviewed changes

slackapi Bot and others added 18 commits August 9, 2026 00:45

feat: add webapi response changes as of 2026-08-09

852a7b6

feat: add webapi response changes as of 2026-08-10

e2dae5e

feat: add webapi response changes as of 2026-08-11

d46be60

feat: add webapi response changes as of 2026-08-11

c935288

feat: add webapi response changes as of 2026-08-12

397f4c0

feat: add webapi response changes as of 2026-08-12

a934b88

feat: add mcp_slack_activity_list_tool_called audit action

3588548

Detected as a new audit action by the E2E runner (Unknown action detected). Adds the constant alongside the other mcp_slack_*_tool_called entries. ❤️ Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

feat: add webapi response changes as of 2026-08-12

f51b8ba

feat: add webapi response changes as of 2026-08-13

c130876

feat: add webapi response changes as of 2026-08-13

763b225

feat: add webapi response changes as of 2026-08-13

3258e45

feat: add webapi response changes as of 2026-08-13

1d32cbe

slackapi Bot and others added 3 commits August 14, 2026 00:49

feat: add webapi response changes as of 2026-08-14

7d97b0e

feat: add webapi response changes as of 2026-08-14

3fd982b

zimeg reviewed Aug 14, 2026

View reviewed changes

zimeg and others added 2 commits August 14, 2026 15:58

feat: add webapi response changes as of 2026-08-14

854c57e

Navigation Menu

Uh oh!

feat: add webapi response changes as of 2026-08-07 - #1629

feat: add webapi response changes as of 2026-08-07#1629
slackapi[bot] wants to merge 30 commits into
mainfrom
api

slackapi Bot commented Aug 7, 2026

Uh oh!

codecov Bot commented Aug 7, 2026 •

edited

Loading

Uh oh!

zimeg left a comment

Uh oh!

zimeg Aug 8, 2026

Uh oh!

zimeg left a comment

Uh oh!

zimeg Aug 14, 2026

Uh oh!

zimeg Aug 14, 2026

Uh oh!

zimeg Aug 14, 2026

Uh oh!

zimeg Aug 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

slackapi Bot commented Aug 7, 2026

Category

Requirements

Uh oh!

codecov Bot commented Aug 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

zimeg Aug 8, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg left a comment

Choose a reason for hiding this comment

Uh oh!

zimeg Aug 14, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg Aug 14, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg Aug 14, 2026

Choose a reason for hiding this comment

Uh oh!

zimeg Aug 14, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

codecov Bot commented Aug 7, 2026 •

edited

Loading